Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.58%. Comparing base (3771a0a) to head (7991dbc).
⚠️ Report is 21 commits behind head on main.

@@           Coverage Diff           @@
##             main     #987   +/-   ##
=======================================
  Coverage   66.58%   66.58%           
=======================================
  Files          40       40           
  Lines        6057     6057           
  Branches     1014     1014           
=======================================
  Hits         4033     4033           
  Misses       1663     1663           
  Partials      361      361           

1. I would add a FutureWarning about this behavior
2. That sounds reasonable to make a folllow-up issue. Let's try to lay out everywhere this sort of thing occurs and then we can proceed with which ones can be removed (and given FutureWarnings)
3. No need to ping, a review is fine :)

@ilan-gold I could send a future warning but are we going to deprecate something else if we are going to replace parallelize? I feel like we should throw warnings if we will surely replace it because many functions seem to use parallelize and we might be sending annoying warnings for a long time. Maybe I can also just remove the sentence where I say n_jobs will be removed in the future and just mention that it us to numba ultimately how many cores will be utilized.

@ilan-gold I also don't understand, I think @Intron7 had an opinion on the parallelize utility function. If it's worth refactoring maybe we should consider but maybe we can merge this if thats still fine